Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a script to run edkrepo_cli.py without edkrepo installed to host. #242

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

ndhaller
Copy link
Contributor

As the edkrepo installer is not run on host system, include a requirements file for the python requirements.

@ndhaller ndhaller marked this pull request as ready for review May 10, 2024 00:18
@JFDiaz
Copy link

JFDiaz commented May 30, 2024

Works beautifully. I see no reason why we need an OS specific installer for the edkrepo tool.

ndhaller added 4 commits July 25, 2024 16:35
As the edkrepo installer is not run, include a requirements file for the python requirements.

Signed-off-by: Nathaniel Haller <[email protected]>
Running edkrepo_dev.py from the source repo directly creates __pycache__ folders.
Create a .gitignore file to ignore those folders.

Signed-off-by: Nathaniel Haller <[email protected]>
…sts.

When running edkrepo_dev.py, if no other global config directory is found,
edkrepo_dev_local_data directory relative to the edkrepo_dev.py script
becomes the default.

All config files and manifest repos added to the edkrepo_dev_local_data
directory by edkrepo_dev.py will be ignored by git.

Use of the edkrepo_dev.py edkrepo_dev_local_data directory is limited to 'win32' platforms.

Signed-off-by: Nathaniel Haller <[email protected]>
If the cfg file contains command-packages section includes multiple packages,
raise ImportError if all packages fail to import.

GlobalConfig file command-packages can be expanded and remain testable with
with Tianocore edk2-edkrepo edkrepo.commands.

Signed-off-by: Nathaniel Haller <[email protected]>
@ndhaller
Copy link
Contributor Author

Rebased the branch to Version 3.2.4 as the base.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants